Skip to content

feat(UI): implement SetMachineParameter methods - #72

Closed
krixkrix wants to merge 1 commit into
alexanderoster:developfrom
krixkrix:extended_ui_environment_api
Closed

krixkrix wants to merge 1 commit into
alexanderoster:developfrom
krixkrix:extended_ui_environment_api

Conversation

@krixkrix

@krixkrix krixkrix commented Feb 5, 2026

Copy link
Copy Markdown

Adding four new methods to the UIEnvironment to allow direct parameter updates from UI event handlers:

  • SetMachineParameter
  • SetMachineParameterAsDouble
  • SetMachineParameterAsInteger
  • SetMachineParameterAsBool

Adding four new methods to the UIEnvironment to allow direct parameter updates from UI event handlers:
- SetMachineParameter
- SetMachineParameterAsDouble
- SetMachineParameterAsInteger
- SetMachineParameterAsBool

Ported onto upstream develop: develop added the four
GetMachineParameterGroupParameter* getters at the same spot in
ACT/LibMCEnv.xml and libmcenv_uienvironment.hpp; both sets are kept, the
setters placed after the new getters. Framework files were regenerated
with ACT rather than merged. The setters go through
CParameterGroup::set*ParameterValueByName, which on develop bumps the
state machine's frontend change counter, so SSE live updates fire when a
UI event sets a machine parameter.
@krixkrix
krixkrix force-pushed the extended_ui_environment_api branch from ca5211b to 7b7e423 Compare October 9, 2026 13:17
@krixkrix

Copy link
Copy Markdown
Author

For now rejected by @alexanderoster, quoting:
The actually intent of not setting machine parameters from UI events was to force people into keeping a thought on the cleanliness of the data flow.

@krixkrix krixkrix closed this Oct 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant